-
Notifications
You must be signed in to change notification settings - Fork 5
cmsis_6: Integrate with zephyr build system #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cmsis_6: Integrate with zephyr build system #1
Conversation
Hi @RobinKastberg, could you please add more details in the commit message like what the change is and why is it needed? Also, in the PR description, please add a link to the Zephyr PR that validates these changes in the Zephyr CI infrastructure. Please have a look at https://docs.zephyrproject.org/latest/develop/modules.html#process-for-submitting-changes-to-existing-modules for more details. |
e82c5b3
to
23c5406
Compare
ARM has deprecated 5.9.0 and we will need to start using CMSIS_6 to get upstream features and continue supporting tf-m. This commit will integrate it just like CMSIS_5. Signed-off-by: Robin Kastberg <[email protected]>
23c5406
to
d9285bd
Compare
Thanks! I hadn't read that part of the manual. Here is the zephyr PR: |
It looks like we are missing zephyrproject-rtos/cmsis@54eaf19 |
Using this PR for switching TF-M and Cortex-M to CMSIS_6, Zephyr PR that uses this is : zephyrproject-rtos/zephyr#89370 |
@RobinKastberg I was not able to push a change to your PR so created another one here. I'll follow up progress on that for convenience. |
These changes are merged via another PR so closing this. |
Add the cmsis_5 build system to cmsis_6
Someone more knowledgable then me needs to check this.
PS: IAR has a CRITICAL bug in CMSIS 5.9.0 so to be able to support zephyr (coming shortly) we really really need CMSIS_6 or backport the fix into CMSIS 5.9.0.
Zephyr PR: zephyrproject-rtos/zephyr#84330